Skip to content

Fix rebase conflict detection in git 2.26#10929

Merged
lafriks merged 3 commits into
go-gitea:masterfrom
zeripath:fix-rebase-conflicts-code-for-git-2.26
Apr 3, 2020
Merged

Fix rebase conflict detection in git 2.26#10929
lafriks merged 3 commits into
go-gitea:masterfrom
zeripath:fix-rebase-conflicts-code-for-git-2.26

Conversation

@zeripath
Copy link
Copy Markdown
Contributor

@zeripath zeripath commented Apr 2, 2020

Git changed the technique used in rebase from simple apply-patches to use merge. This breaks our conflict detection code.

Fix #10928

Signed-off-by: Andrew Thornton art27@cantab.net

Git changed the technique used in rebase from
simple apply-patches to use merge. This breaks
our conflict detection code.

Signed-off-by: Andrew Thornton <art27@cantab.net>
@zeripath zeripath added this to the 1.12.0 milestone Apr 2, 2020
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Apr 2, 2020
Comment thread services/pull/merge.go Outdated
Signed-off-by: Andrew Thornton <art27@cantab.net>
@6543
Copy link
Copy Markdown
Member

6543 commented Apr 2, 2020

@zeripath please send second patch ;)

@zeripath
Copy link
Copy Markdown
Contributor Author

zeripath commented Apr 3, 2020

@6543 So I think the second commit was sent by the time you commented.

@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Apr 3, 2020
@lafriks lafriks merged commit 97eb37e into go-gitea:master Apr 3, 2020
@lafriks lafriks added the backport/done All backports for this PR have been created label Apr 3, 2020
@zeripath zeripath deleted the fix-rebase-conflicts-code-for-git-2.26 branch April 3, 2020 16:22
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

backport/done All backports for this PR have been created lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Support git v2.26.0

5 participants